Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove now-unused channel errors #3713

Merged
merged 1 commit into from
Dec 3, 2024
Merged

Conversation

benclifford
Copy link
Collaborator

The final use of channel errors was removed in PR #3690

This is part of #3515 channel removal project

Changed Behaviour

Any user catching these errors explicitly will now get an error about the exceptions being undefined. Because these don't exist any more, it should be safe to remove those catches.

Type of change

  • Code maintenance/cleanup

The final use of channel errors was removed in PR #3690
Copy link
Collaborator

@khk-globus khk-globus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️

@khk-globus khk-globus added this pull request to the merge queue Dec 3, 2024
Merged via the queue into master with commit a9a5b4b Dec 3, 2024
7 checks passed
@khk-globus khk-globus deleted the benc-3515-remove-channel-errors branch December 3, 2024 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants